Skip to content

Conversation

@rafaelroquetto
Copy link
Contributor

Ensure that Makefile target only fails if there's a problem with go.mod and go.sum. The current implementation will fail if anything in the tree is different (such as unstaged files), making it harder to pinpoint what and where the true failure lies.

@rafaelroquetto rafaelroquetto requested a review from a team as a code owner November 29, 2025 01:35
@codecov
Copy link

codecov bot commented Nov 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 47.20%. Comparing base (67ae3cc) to head (8aa2935).

❗ There is a different number of reports uploaded between BASE (67ae3cc) and HEAD (8aa2935). Click for more details.

HEAD has 43 uploads less than BASE
Flag BASE (67ae3cc) HEAD (8aa2935)
oats-test 5 0
k8s-integration-test 15 0
integration-test-vm-${ARCH}-${KERNEL_VERSION} 12 0
integration-test 10 0
integration-test-arm 1 0
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #951      +/-   ##
==========================================
- Coverage   55.84%   47.20%   -8.65%     
==========================================
  Files         255      255              
  Lines       22152    22152              
==========================================
- Hits        12371    10456    -1915     
- Misses       8942    11026    +2084     
+ Partials      839      670     -169     
Flag Coverage Δ
integration-test ?
integration-test-arm ?
integration-test-vm-${ARCH}-${KERNEL_VERSION} ?
k8s-integration-test ?
oats-test ?
unittests 47.20% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant